황성민 과제제출합니다. #1

hll2071 · 2024-07-10T10:01:31Z

No description provided.

jyj1289

너무 수고하셨습니다!
대체적으로 문제 이해를 잘못 이해하신 것 같아서 그에 대한 리뷰를 달아드렸습니다.
코드의 원리나 작동 방식등을 이해하고 코드를 작성하시면 훨씬 더 양질의 코드가 나올거에요. 너무 수고많으셨고 리뷰를 읽어보시고 수정해주시길 바라겠습니다 :)

jyj1289 · 2024-07-11T04:52:07Z

Main.java

+        for(int i=0; i<attackerlist.size(); i++) {
+            for(String shoot : attackerlist.get(i).ball) {
+                for(String goal : goalkeeperlist.get(0).keep) {
+                    if(shoot.contains(goal)) miss[i] +=1;


삼중 for문을 사용해야할 이유가 있을까요? 차라리 goalkeeper의 keep을 for문으로 두고 attacker 두 명의 슛과 비교하는게 낫지 않을까요?

contains()의 원리에 대해 생각해 보세요.

jyj1289 · 2024-07-11T04:53:03Z

Main.java

+        int[] miss = new int[2];
+        for(int i = 0; i < miss.length; i++) {
+            miss[i] = 0;
+        }


만약 배열을 0으로 초기화하고 싶으시다면 int[] miss = new int[]{0, 0};을 사용하여 선언하면서 초기화할 수 있습니다.

jyj1289 · 2024-07-11T05:00:53Z

Main.java

+            balls[i] = sc.nextLine();
+            attackerlist.get(i).ball = balls[i].split(",");
+        }


attackerlist.get(i).ball =sc.nextLine().split(",");
로 줄일 수 있습니다. 또한, 문제에서는 attackerlsit로 받지 않는다는 것을 명심해주세요.

또한, attackerlist는 카멜케이스를 위배합니다.

jyj1289 · 2024-07-11T05:01:53Z

Main.java

+        }
+        String[] balls = new String[2];
+        String keepball;
+        Attacker[] attackers = new Attacker[2];


attacker1과 attacker2를 각각 ArrayList로 선언하라고 되어있습니다.

jyj1289 · 2024-07-11T05:02:30Z

Main.java

+        for(int i = 0; i < attackers.length; i++) {
+            attackers[i] = new Attacker();
+        }
+        ArrayList<Goalkeeper> goalkeeperlist = new ArrayList<Goalkeeper>();


goalkeeper 또한 goalkeeperlist가 아닌 ArrayList로 선언하라고 되어있습니다.

jyj1289 · 2024-07-11T05:03:02Z

Main.java

+class Attacker {
+    String name;
+    String[] ball = new String[5];
+}
+class Goalkeeper {
+    String[] keep = new String[4];
+}


class가 아닌 attacker1과 attacker2, goalkeepr를 ArrayList로 선언하라고 되어있습니다.

jyj1289 · 2024-07-11T05:05:04Z

Main.java

+        goalkeeperlist.add(new Goalkeeper());
+        ArrayList<Attacker> attackerlist = Arrays.stream(attackers).collect(Collectors.toCollection(ArrayList::new));


이러한 코드들이 문제를 잘못 이해하여 생긴 것이죠.

jyj1289 · 2024-07-11T05:06:54Z

Main.java

+        keepball = sc.nextLine();
+
+        goalkeeperlist.get(0).keep = keepball.split(",");
+        for(int i=0; i<attackerlist.size(); i++) {


attackerList를 사용하지 않았더라면 위와 같은 쓸모 없는 for문이 감소했겠죠.

jyj1289 · 2024-07-11T05:07:40Z

Main.java

+        if(miss[0]>miss[1]) System.out.println(attackerlist.get(1).name + "님이 승리하셨습니다.");
+        else if (miss[0]<miss[1]) System.out.println(attackerlist.get(0).name + "님이 승리하셨습니다.");
+        else System.out.println("무승부입니다.");
+    }
+}


여기도 attackerlist가 아닌 attacker1과 attacker2라는 ArrayList를 생성한 뒤 각각처리해주세요.

init

9b58999

jyj1289 reviewed Jul 11, 2024

View reviewed changes

perf : 첨부터 다시 작성

7a503ec

		goalkeeperlist.add(new Goalkeeper());
		ArrayList<Attacker> attackerlist = Arrays.stream(attackers).collect(Collectors.toCollection(ArrayList::new));

황성민 과제제출합니다. #1

Are you sure you want to change the base?

황성민 과제제출합니다. #1

Uh oh!

Conversation

hll2071 commented Jul 10, 2024

Uh oh!

jyj1289 left a comment

Choose a reason for hiding this comment

Uh oh!

jyj1289 Jul 11, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

jyj1289 Jul 11, 2024 •

edited

Loading